Skip to content

Cto firmware#284

Open
aaumar25 wants to merge 16 commits into
chain_commandsfrom
CTO_firmware
Open

Cto firmware#284
aaumar25 wants to merge 16 commits into
chain_commandsfrom
CTO_firmware

Conversation

@aaumar25

@aaumar25 aaumar25 commented Jun 22, 2026

Copy link
Copy Markdown
Member

Commands to test:

  • HELP [command]
  • VERSION
  • LOAD_CONFIG [file path]
  • WAIT (duration)
  • CLEAR
  • SET (variable) (value)
  • GET (variable)
  • VARIABLES
  • TIMER_START
  • TIMER_READ --> see Timer_Read correct time display #294
  • FILE (path)
  • SAVE_OUTPUT (mode) [path]
  • EXIT
  • MCL_VERSION --> see MCL_VERSION shows mmc-api version instead of cc-link version #288
  • CONNECT
  • DISCONNECT
  • SET_SPEED (line name) (speed percentage)
  • SET_ACCELERATION (line name) (acceleration percentage)
  • GET_SPEED (line name)
  • GET_ACCELERATION (line name)
  • PRINT_X (line name) (axis)
  • PRINT_Y (line name) (axis)
  • PRINT_WR (line name) (axis)
  • PRINT_WW (line name) (axis)
  • AXIS_SLIDER (line name) (axis) [result variable]
  • SLIDER_LOCATION (line name) (slider) [result variable]
  • SLIDER_AXIS (line name) (slider)
  • HALL_STATUS (line name) [axis] --> closes HALL_STATUS command ignores axis parameter #287, closed by 8a25a09 commit
  • ASSERT_HALL (line name) (axis) (side) [on/off] --> closes ASSERT_HALL always return InvalidCharacter #289, closed by 610ed11
  • CLEAR_ERRORS (line name) (axis)
  • CLEAR_SLIDER_INFO (line name) (axis)
  • RELEASE_AXIS_SERVO (line name) (axis) --> see RELEASE_AXIS_SERVO hangs and disallow further command #290
  • STOP_TRAFFIC (line name) (axis) (direction) --> doesn't seem to do anything
  • ALLOW_TRAFFIC (line name) (axis) (direction)
  • CALIBRATE (line name)
  • HOME_SLIDER (line name) --> This command is not intended for the current type of machine (for carrier with linear motor machines)
  • WAIT_HOME_SLIDER (line name) [result variable] --> This command is not intended for the current type of machine (for carrier with linear motor machines)
  • ISOLATE (line name) (axis) (direction) [slider id] [link axis]
  • RECOVER_SLIDER (line name) (axis) (new slider ID) [use sensor]
  • WAIT_RECOVER_SLIDER (line name) (axis) [result variable]
  • MOVE_SLIDER_AXIS (line name) (slider) (destination axis) --> closes MOVE_SLIDER_XXX command unexpected behavior when slider in between two drivers #291 by 4201e53 & MOVE_SLIDER_XXX command cannot finish movement when moving backward #292 is closed by recalibration
  • MOVE_SLIDER_LOCATION (line name) (slider) (destination location)
  • MOVE_SLIDER_DISTANCE (line name) (slider) (distance)
  • SPD_MOVE_SLIDER_AXIS (line name) (slider) (destination axis)
  • SPD_MOVE_SLIDER_LOCATION (line name) (slider) (destination location)
  • SPD_MOVE_SLIDER_DISTANCE (line name) (slider) (distance)
  • WAIT_MOVE_SLIDER (line name) (slider)
  • PUSH_SLIDER_FORWARD (line name) (slider)
  • PUSH_SLIDER_BACKWARD (line name) (slider)
  • PULL_SLIDER_FORWARD (line name) (axis) (slider) (location) --> Continuous pull is only implemented in John's firmware. I am not sure if CTO's firmware has it. 이명현 requests it but it is just impossible to implement. However, he showed that the pull slider has error. EDIT: CTO added support for location in pull. closes Pull command behaves incorrectly #286 by 36dce60 commit
  • PULL_SLIDER_BACKWARD (line name) (axis) (slider) (location) --> closes issue Pull command behaves incorrectly #286 by 36dce60 commit
  • WAIT_PULL_SLIDER (line name) (axis)
  • STOP_PULL_SLIDER (line name) (axis)
  • SLIDER_CHAIN_LINK (line name) (first axis) (second axis) --> panics
  • SLIDER_CHAIN_UNLINK (line name) (first axis) (second axis)
  • SET_LEFT_CHAIN_ON (line name) (axis)
  • SET_RIGHT_CHAIN_ON (line name) (axis)
  • SET_LEFT_CHAIN_OFF (line name) (axis)
  • SET_RIGHT_CHAIN_OFF (line name) (axis)
  • MOVE_SLIDER_CHAIN (line name) (head slider ID) (destination axis)

@aaumar25

aaumar25 commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

FIxed mock build on latest commit. Solution: module for executable and module for test must be a different module.

cc: @solara404

@aaumar25 aaumar25 linked an issue Jun 24, 2026 that may be closed by this pull request
@aaumar25

Copy link
Copy Markdown
Member Author

@solara404 Only release_axis_servo and mcl_version left. I pass the ownership of this PR and the PR on mmc-api repo to you for merging. If later you want to add more feature to this branch, it is very welcome. I suggest to re-implement the auto-complete feature from our main branch to this branch as I believe this feature will be pretty much appreaciated by anyone who use this cli.

Tag me if you need review for any PR or any questions you have regarding CTO's firmware! I will be glad to assist you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ASSERT_HALL always return InvalidCharacter HALL_STATUS command ignores axis parameter Pull command behaves incorrectly

2 participants